Skip to content

Added AnalysisMessage component to imagine25#1461

Merged
JackyFTW merged 5 commits into1364-epic-imagine-2025-changesfrom
1398-story-imagine-25---add-analysis-message-component
Apr 7, 2026
Merged

Added AnalysisMessage component to imagine25#1461
JackyFTW merged 5 commits into1364-epic-imagine-2025-changesfrom
1398-story-imagine-25---add-analysis-message-component

Conversation

@JackyFTW
Copy link
Copy Markdown
Member

@JackyFTW JackyFTW commented Apr 2, 2026

Added AnalysisMessage component to the image25/components folder. Imported prior analysis messages and improved styling. Now appears below the video component after Galaga. Prevents users from continuing until they acknowledge the message.

  • No discrepancies across browsers (ex: chrome vs safari)
  • Accessibility functions
  • Pages can scale without distorting page
  • No dead links
  • Navbar is consistent across the site
  • Pages are screen-reader accessible
  • Contrast meets standards for accessibility
  • Pages are keyboard accessible
  • Code is cleaned up and bug-free (ex: debug statements removed)

@JackyFTW JackyFTW requested review from ejt7508 and wlh3903 April 2, 2026 19:58
@JackyFTW JackyFTW self-assigned this Apr 2, 2026
Copy link
Copy Markdown
Contributor

@ejt7508 ejt7508 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct spelling of "survey" in "Congrats on winning! You may collect a prize after completing the post survery for being so awesome sauce."

Comment thread client/src/components/imagine25/pages/Analysis.js Outdated
JackyFTW added 2 commits April 7, 2026 11:34
… of github.com:all-rit/ALL into 1398-story-imagine-25---add-analysis-message-component
@JackyFTW JackyFTW merged commit 2830a48 into 1364-epic-imagine-2025-changes Apr 7, 2026
@JackyFTW JackyFTW deleted the 1398-story-imagine-25---add-analysis-message-component branch April 7, 2026 15:36
@JackyFTW JackyFTW linked an issue Apr 9, 2026 that may be closed by this pull request
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Story: Imagine 25 - Add Analysis Message Component

2 participants